-
Notifications
You must be signed in to change notification settings - Fork 43.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Possible workaround for #796 #1302
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hope it's useful
@esurharun There are conflicts now |
Co-authored-by: Fabian Dill <Berserker66@users.noreply.github.com>
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
3 similar comments
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
@esurharun Sorry it's our bot went crazy. We'll sort it out. |
Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly. |
Can't wait for this to be approved. Huge blocker to awesome things :-) |
Fixed in #2542. Thanks for submitting though! |
Background
Please check out issue #796
Changes
Just applied OpenAI cookbook at https://github.com/openai/openai-cookbook/blob/main/examples/Embedding_long_inputs.ipynb
PR Quality Checklist
I just coded to speed up to the full fix. It needs to get revized and considered for Azure option too.